meta: update pnpm from 11.25.0 to 12.8.1 - #9181
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
araujogui
left a comment
araujogui
left a comment
There was a problem hiding this comment.
LGTM although bot is missing new version on PR title and description.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #9181 +/- ##
=======================================
Coverage 86.15% 86.15%
=======================================
Files 86 86
Lines 6051 6051
Branches 357 357
=======================================
Hits 5213 5213
Misses 834 834
Partials 4 4 ☔ View full report in Codecov by Harness. |
MattIPv4
changed the title
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The major-version upgrade needs successful cross-platform build and browser-test confirmation before approval.
Review effort: Balanced
Findings: None
What changed in this PR
Updates the repository’s package manager from pnpm 11.25.0 to 12.8.1.
Changes:
- Aligns both package-manager version declarations.
- Refreshes the lockfile’s pnpm dependencies and platform binaries.
| File | Description |
|---|---|
pnpm-lock.yaml |
Updates pnpm and its platform-specific dependency records. |
package.json |
Pins pnpm 12.8.1 in packageManager and devEngines. |
Files not reviewed (1)
- pnpm-lock.yaml: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
📦 Build Size ComparisonSummary
|
Updates pnpm from 11.25.0 to
cc @nodejs/web-infra
Check this workflow's logs at https://github.com/nodejs/nodejs.org/actions/runs/36799361888.